Skip to content

Conversation

@Grillo-0
Copy link
Member

Change the contributing rules to make it less cumbersome and specify the style of commit messages.

@Grillo-0 Grillo-0 marked this pull request as draft January 30, 2025 19:43
@Grillo-0 Grillo-0 force-pushed the contrib-docs-refactor branch 13 times, most recently from 23d4f6e to 7420240 Compare January 30, 2025 20:28
@Grillo-0 Grillo-0 force-pushed the contrib-docs-refactor branch from 7420240 to 1baf375 Compare June 6, 2025 03:40
@Grillo-0 Grillo-0 requested a review from monacofj June 6, 2025 03:41
@Grillo-0 Grillo-0 marked this pull request as ready for review June 14, 2025 19:09
@Grillo-0 Grillo-0 self-assigned this Jun 29, 2025
@Grillo-0 Grillo-0 force-pushed the contrib-docs-refactor branch from 1baf375 to 2364d26 Compare September 24, 2025 00:46
@Grillo-0 Grillo-0 requested review from JeanJPNM and artP2 September 24, 2025 00:46
For standardization and simplicity let's use the more common header
notation style.
Indentation in this case just makes it harder to read and write the raw
file, just remove it.
The list has a wrong item syntax on the end and also doesn't have a
link to the files.
I realy don't care how you name your PR branches, they are ephemeral.
I think these tags fill up the first commit line than they give some
useful information. Also the commit message guidelines are not
descriptive enough.

Try to write a better guideline following this great blogpost[1].

[1]: https://cbea.ms/git-commit/
@Grillo-0 Grillo-0 force-pushed the contrib-docs-refactor branch from 2364d26 to c446e62 Compare September 24, 2025 00:52
I don't think is necessary to create an issue before doing an PR, it
just make the contribution process more cumbersome.
We don't need this exception, every contribution must pass the same
process.
Be more specific about what is the target branch to use. And also write
that the tests are obligatory for a PR to be accepted.
Markdown has a feature to make links, let's use it instead of having a
list of references.
This is done to remove redundant information.
@Grillo-0 Grillo-0 force-pushed the contrib-docs-refactor branch from c446e62 to fbdface Compare September 24, 2025 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants